Skip to content

Add functionality for including environment variables as metadata #771

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 8 commits into from
Aug 14, 2025

Conversation

kzscisoft
Copy link
Collaborator

@kzscisoft kzscisoft commented Mar 26, 2025

Environment Variables as Metadata

Issue: #726

Python Version(s) Tested: 3.13

Operating System(s): Ubuntu 24.10

Documentation PR: N/A

📝 Summary

Allows users to specify through globular expressions within their config file or in run.init(...) any environment variables to store as metadata.

🔄 Changes

  • Modifications to the run component of the user configuration, adding the option record_shell_vars, a set of globular expressions or None.
  • Modification to Run.init method to have parameter record_shell_vars.

✔️ Checklist

  • Unit and integration tests passing.
  • Pre-commit hooks passing.
  • Quality checks passing.
  • Updated the documentation.

@kzscisoft kzscisoft self-assigned this Aug 11, 2025
@kzscisoft kzscisoft force-pushed the feature/env-var-metadata branch from cede0e6 to 45072be Compare August 11, 2025 12:47
@kzscisoft kzscisoft added this to the Python API v2.2 milestone Aug 11, 2025
@kzscisoft kzscisoft assigned alahiff and unassigned kzscisoft Aug 11, 2025
Copy link
Collaborator

@alahiff alahiff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🍏 Successfully tested

@kzscisoft kzscisoft merged commit fe69caf into dev Aug 14, 2025
20 checks passed
@kzscisoft kzscisoft deleted the feature/env-var-metadata branch August 14, 2025 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Automatically include SLURM_* environment variables in metadata
2 participants